-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix/external upload limit #6556
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Fix/external upload limit #6556
Conversation
|
@nicolas-raoul @RitikaPahwa4444 Any update on this PR ? |
|
Hi @nicolas-raoul, just checking in to see if this PR is still under review. Please let me know whenever you have time if there’s anything I should update or change. Thanks! |
|
Sorry I am super busy this start of year, hopefully someone else can review? 😥 |
|
No worries at all @nicolas-raoul, totally understand 😊 |
|
✅ Generated APK variants! |
This version is enhanced from the initial proposal (#6555) and includes key refactorings:
A non-blocking toast is shown to the user informing them that only the first 20 files were loaded, ensuring workflow continuity with clear feedback.
Both UploadActivity.kt and CustomSelectorActivity.kt now correctly reference this centralized constant, improving code maintainability and adherence to DRY principles.